WIP: Technical blog post on package installation#17442
Conversation
Documentation ReviewI've reviewed this WIP technical blog post on package installation. Here are my findings: Issues FoundLine 21: TODO placeholder linksTwo TODO placeholders need to be replaced with actual documentation links:
Suggestion: These should link to actual documentation. If the documentation doesn't exist yet, consider either creating it first or linking to general plugin documentation at Line 32: Typo - "then" should be "than"Line 29: Line 84: Syntax error - extra backtickLine 84: Line 180: Content appears incompleteThe section "Mutable DAG traversal" (line 174) ends with just "..." suggesting the content is unfinished. Since this is marked as WIP, this is expected, but it's the main technical content that readers will be looking for. Positive Observations
Recommendations for Completion
Style Compliance
Once the content is complete and the TODOs are resolved, this will be a solid technical deep-dive that complements the user-facing documentation. Mention @claude if you'd like additional reviews or fixes after addressing these issues. |
|
Your site preview for commit 7fcd7e4 is ready! 🎉 http://www-testing-pulumi-docs-origin-pr-17442-7fcd7e4b.s3-website.us-west-2.amazonaws.com. |
Fixes https://github.com/pulumi/home/issues/4502